London | 25-ITP-Sep | Zadri Abdule | Sprint 1 | Coursework#837

Closed

Zadri415 wants to merge 15 commits into

CodeYourFuture:mainfrom

Zadri415:coursework/sprint-1

Zadri415 commented Nov 1, 2025 •

edited

Loading

Learners, PR Template

Self checklist

I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
My changes meet the requirements of the task
I have tested my changes
My changes follow the style guide

Changelist

Completed all of the tasks for Sprint 1 of Module Structuring & Testing Data.

Questions

Ask any questions you have for your reviewer.
Not at the moment.

Zadri415 added 13 commits

October 14, 2025 20:50


          Explain assignment in count increment example

7995c8d


          Add intitials variables charAt method

5213e77


          Add dir and ext variables

c77a3cf


          Add explanatory comments to random number generator code

ba118b5


          Comment out instruction lines so they are not executed

d5dfe95


          Change age variable from const to let to allow reassignment

5c9289a


          Add comment explaining instialization error

8f743b2


          Add predict that the code will fail because .slice() is not valid on …

095d155

…a number


          Predict that the code will fail because .slice() is not valid on a nu…

e054a32

…mber


           Fix: renamed variable starting with a number to follow JS naming rules

6ea69f1


          Added MDN-based explanation,and documented variable declarations

40825f8


          Completed answers for 2-time-format.js

5f00dde


          Added line-by-line explanations for 3-to-pounds.js

265ee10

github-actions Bot commented Nov 1, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

Zadri415 added the Needs Review label

github-actions Bot commented Nov 1, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

Zadri415 added the Module-Structuring-And-Testing-Data label

github-actions Bot commented Nov 1, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

Zadri415 changed the title ~~London | 25-ITP-Sep | Zadri Abdule | Sprint 1 | Coursework~~ London | 25-ITP-Sep | Zadri Abdule | Sprint 1 | Module Structuring & Testing Data

github-actions Bot commented Nov 1, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

3 similar comments

github-actions Bot commented Nov 1, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions Bot commented Nov 2, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions Bot commented Nov 2, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

Zadri415 removed the Module-Structuring-And-Testing-Data label

github-actions Bot commented Nov 2, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).


          Merge branch 'main' into coursework/sprint-1

LonMcGregor added the Review in progress label

github-actions Bot commented Nov 3, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

LonMcGregor reviewed

View reviewed changes

LonMcGregor left a comment

Just one small thing to fix in file 1.js

Sprint-1/2-mandatory-errors/1.js

LonMcGregor removed the Needs Review label

LonMcGregor added Reviewed and removed Review in progress labels

github-actions Bot commented Nov 3, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

2 similar comments

github-actions Bot commented Nov 3, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions Bot commented Nov 3, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).


          Update 1.js

0d51c23

Zadri415 changed the title ~~London | 25-ITP-Sep | Zadri Abdule | Sprint 1 | Module Structuring & Testing Data~~ London | 25-ITP-Sep | Zadri Abdule | Sprint 1 | Coursework

github-actions Bot commented Nov 10, 2025

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

illicitonion closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels